Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Slayer System rework #1040

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
Open

Slayer System rework #1040

wants to merge 19 commits into from

Conversation

7azeemm
Copy link
Contributor

@7azeemm 7azeemm commented Nov 1, 2024

  • Added Slayer HUD
  • Added Boss slain time
  • Added Personal Best slain time
  • Added Boss and MiniBoss spawn alert
  • Added Mute Enderman sounds
  • Added Lazer phase Timer

@7azeemm
Added Slayer HUD
8d04e75 
Added Boss slain time
Added Personal Best slain time
Added Boss and MiniBoss spawn alert
Added Mute Enderman sounds
Added Lazer phase Timer
@LifeIsAParadox LifeIsAParadox added the reviews needed This PR needs reviews label Nov 1, 2024
@7azeemm
Copy link
Contributor Author

7azeemm commented Nov 1, 2024

image
just need help fixing this
i didn't figure out how to make text renders behind entity
also glow effect is broken (filled not outline like in img) in beta 1 for only 1.21.3 minecraft version, beta 1 1.21.1 works fine. (same problem without sodium)

idk if that's a good color combo in hud
i'm bad at designing
image

@Emirlol
Copy link
Collaborator

Emirlol commented Nov 1, 2024

Looks pretty good.
I went ahead and fixed the filled outline issue, along with a few other things that kept crashing my game so that I could actually test it. Hope you don't mind.

@7azeemm
Copy link
Contributor Author

7azeemm commented Nov 1, 2024

nice, appreciate your help

Emirlol
Emirlol requested changes Nov 1, 2024
View reviewed changes
Copy link
Collaborator

@Emirlol Emirlol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few nitpicks here and there but the code looks good in general. I didn't look too deep into the logic since stuff just seemed to work, excluding the LazerTimer as I wasn't able to test it. (too weak for voidgloom seraph lol)

src/main/java/de/hysky/skyblocker/skyblock/slayers/SlayerManager.java Outdated Show resolved Hide resolved
src/main/java/de/hysky/skyblocker/skyblock/slayers/SlayerManager.java Outdated Show resolved Hide resolved
src/main/java/de/hysky/skyblocker/skyblock/slayers/SlayerManager.java Outdated Show resolved Hide resolved
src/main/java/de/hysky/skyblocker/skyblock/slayers/SlayerManager.java Outdated Show resolved Hide resolved
src/main/java/de/hysky/skyblocker/skyblock/slayers/SlayerManager.java Outdated Show resolved Hide resolved
src/main/java/de/hysky/skyblocker/skyblock/slayers/hud/SlayerHudWidget.java Outdated Show resolved Hide resolved
src/main/java/de/hysky/skyblocker/skyblock/slayers/SlayerManager.java Outdated Show resolved Hide resolved
src/main/resources/assets/skyblocker/lang/en_us.json Outdated Show resolved Hide resolved
src/main/resources/assets/skyblocker/lang/en_us.json Outdated Show resolved Hide resolved
src/main/resources/skyblocker.mixins.json Outdated Show resolved Hide resolved
@LifeIsAParadox LifeIsAParadox added changes requested This PR need changes and removed reviews needed This PR needs reviews labels Nov 1, 2024
@Emirlol Emirlol added the new feature This issue or PR is a new feature label Nov 1, 2024
kevinthegreat1
kevinthegreat1 reviewed Nov 4, 2024
View reviewed changes
Copy link
Collaborator

@kevinthegreat1 kevinthegreat1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not a complete review yet, but just some comments.

src/main/java/de/hysky/skyblocker/mixins/SoundSystemMixin.java Outdated Show resolved Hide resolved
src/main/java/de/hysky/skyblocker/mixins/WorldRendererMixin.java Outdated Show resolved Hide resolved
src/main/java/de/hysky/skyblocker/mixins/WorldRendererMixin.java Outdated Show resolved Hide resolved
src/main/java/de/hysky/skyblocker/mixins/WorldRendererMixin.java Outdated Show resolved Hide resolved
src/main/java/de/hysky/skyblocker/utils/Utils.java Outdated Show resolved Hide resolved
@LifeIsAParadox LifeIsAParadox added reviews needed This PR needs reviews and removed changes requested This PR need changes labels Nov 4, 2024
@7azeemm
Copy link
Contributor Author

7azeemm commented Nov 9, 2024

Looks good to me

@7azeemm 7azeemm mentioned this pull request Nov 26, 2024
Open
@AzureAaron AzureAaron added the merge conflicts This PR has merge conflicts that need solving. label Nov 27, 2024
@kevinthegreat1
Copy link
Collaborator

This is next on my list. Could you resolve the merge conflicts please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BigloBot BigloBot BigloBot left review comments

@kevinthegreat1 kevinthegreat1 kevinthegreat1 left review comments

@Emirlol Emirlol Emirlol requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
merge conflicts This PR has merge conflicts that need solving. new feature This issue or PR is a new feature reviews needed This PR needs reviews
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@7azeemm @Emirlol @kevinthegreat1 @BigloBot @LifeIsAParadox @AzureAaron